Skip to content

Conversation

@radurentea
Copy link
Collaborator

@radurentea radurentea commented Apr 3, 2025

Description

Added two ESP-IDF Profiles. for two different C/C++ language server (Microsoft and ClangD)
Added documentation about profiles, with a short description and with the steps to be followed to import the esp-idf profile.

Type of change

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

Steps to test this pull request

Check documentation, download the profile from the provided link and import it to Vs Code.

How has this been tested?

As described above

Test Configuration:

  • ESP-IDF Version: doesn't matter
  • OS (Windows,Linux and macOS): Windows

Checklist

  • PR Self Reviewed
  • Applied Code formatting
  • Added Documentation
  • Added Unit Test
  • Verified on all platforms - Windows,Linux and macOS

@radurentea radurentea added this to the v1.10.0 milestone Apr 3, 2025
@radurentea radurentea self-assigned this Apr 3, 2025
@github-actions
Copy link

github-actions bot commented Apr 3, 2025

Download the artifacts for this pull request:
You can test these changes by installing this VSIX by click menu View -> Command Palette..., type Install from VSIX and then select downloaded esp-idf-extension.vsix file to install the extension.

Copy link
Collaborator

@brianignacio5 brianignacio5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this template should be simplified and keep it ESP-IDF specific.

@radurentea radurentea changed the title feat: Add file and documentation [VSC-1561] Add ESP-IDF Profiles Apr 17, 2025
@brianignacio5 brianignacio5 modified the milestones: v1.10.0, v1.11.0 Apr 21, 2025
- Microsoft Profile: Remove C/C++ Theme, C/C++ Extension Pack, CMake Tools
- Clangd Profile: Remove CMake Tolls
Copy link
Collaborator

@brianignacio5 brianignacio5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

@Fabricio-ESP Fabricio-ESP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues.

@radurentea radurentea merged commit 8dc5a1f into master May 15, 2025
7 checks passed
Copy link

@cbarney929 cbarney929 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cleaning hopefully helps

Copy link

@GrumpyFart-sudo GrumpyFart-sudo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

*thumbs up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants